Skip to content

sentry: Enable tracing feature #3919

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 5, 2021
Merged

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Sep 12, 2021

https://github.com/tokio-rs/tracing/releases/tag/tracing-subscriber-0.2.21 enables us to use per-layer event filters, which previously blocked us from using the tracing feature of sentry. This PR enables the feature using the default configuration, which reports all error!() usages as errors and warn!() and info!() as breadcrumbs for associated errors.

@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear A-backend ⚙️ labels Sep 12, 2021
@Turbo87 Turbo87 force-pushed the sentry-tracing branch 2 times, most recently from b1b6843 to db6a254 Compare September 17, 2021 21:47
@bors
Copy link
Contributor

bors commented Sep 19, 2021

☔ The latest upstream changes (presumably #3920) made this pull request unmergeable. Please resolve the merge conflicts.

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change itself looks good but I'm not involved in the sentry part. If there are some agreements with Justin and/or Pietro about the log level, feel free to r=me.

@Turbo87 Turbo87 force-pushed the sentry-tracing branch 2 times, most recently from 18a73bc to 344c753 Compare September 26, 2021 08:34
@bors
Copy link
Contributor

bors commented Sep 29, 2021

☔ The latest upstream changes (presumably #3965) made this pull request unmergeable. Please resolve the merge conflicts.

@Turbo87
Copy link
Member Author

Turbo87 commented Oct 1, 2021

@pietroalbini @jtgeibel any objections on this?

@Turbo87
Copy link
Member Author

Turbo87 commented Oct 5, 2021

apparently not... 😅

@bors r=JohnTitor

@bors
Copy link
Contributor

bors commented Oct 5, 2021

📌 Commit 422c1a2 has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Oct 5, 2021

⌛ Testing commit 422c1a2 with merge 14736ac...

bors added a commit that referenced this pull request Oct 5, 2021
sentry: Enable `tracing` feature

https://github.com/tokio-rs/tracing/releases/tag/tracing-subscriber-0.2.21 enables us to use per-layer event filters, which previously blocked us from using the `tracing` feature of `sentry`. This PR enables the feature using the default configuration, which reports all `error!()` usages as errors and `warn!()` and `info!()` as breadcrumbs for associated errors.
@bors
Copy link
Contributor

bors commented Oct 5, 2021

💥 Test timed out

@Turbo87
Copy link
Member Author

Turbo87 commented Oct 5, 2021

let's try this again after rebasing...

@bors r=JohnTitor

@bors
Copy link
Contributor

bors commented Oct 5, 2021

📌 Commit b2bb38f has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Oct 5, 2021

⌛ Testing commit b2bb38f with merge 20d51c0...

@bors
Copy link
Contributor

bors commented Oct 5, 2021

☀️ Test successful - checks-actions
Approved by: JohnTitor
Pushing 20d51c0 to master...

@bors bors merged commit 20d51c0 into rust-lang:master Oct 5, 2021
@Turbo87 Turbo87 deleted the sentry-tracing branch October 5, 2021 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants